home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c++-part2 / 16168 < prev    next >
Encoding:
Internet Message Format  |  1996-08-05  |  2.5 KB
  1. Path: news.sprintlink.net!datalytics!usenet
  2. From: Rob Stewart <stew@datalytics.com>
  3. Newsgroups: comp.lang.c++
  4. Subject: Re: [HELP!!!] Segmentation Fault!
  5. Date: Tue, 09 Apr 1996 17:55:13 -0400
  6. Organization: Datalytics, Inc
  7. Message-ID: <316ADCC1.629F@datalytics.com>
  8. References: <4k4gja$mm@nuscc.nus.sg>
  9. NNTP-Posting-Host: 204.62.224.71
  10. Mime-Version: 1.0
  11. Content-Type: text/plain; charset=us-ascii
  12. Content-Transfer-Encoding: 7bit
  13. X-Mailer: Mozilla 2.0 (WinNT; I)
  14.  
  15. Virus wrote:
  16. [snip]
  17. > I keep getting segmentation fault in my program, and I have tried using the
  18. > debugger (gdb in unix), but I cannot understand what the debugger is trying
  19. > to say..[snip]
  20. > 
  21. > char* SpaceStudentFile::readNextField()
  22. > {
  23. >   char str[40];
  24. >   int i = 0;
  25. > 
  26. >   // take away starting whitespaces
  27. >   while (str[0] = fgetc(fd)) {
  28. >         if (*str == ' ') continue;
  29. >         if (*str == '\t') continue;
  30. >         if (*str == '\n') continue;
  31. >         if (*str == EOF) return NULL;
  32. >         break;
  33. >   }
  34. > 
  35. >   // read from " to next "
  36. >   if (str[0] == '"') {
  37. >         for (i=0; ((str[i] = fgetc(fd)) != '"'); i++)
  38. >         ;
  39. >         str[i]=0;
  40. >         return str;
  41. >   }
  42. > 
  43. >   // else read until next whitespace
  44. >   for (i=1; ((str[i] = fgetc(fd)) != ' ')
  45. >         && (str[i] != '\n') && (str[i] != '\t'); i++)
  46. >   ;
  47. >   str[i] = 0;
  48. >   return str;
  49. > 
  50. > }
  51. > 
  52.  
  53. Your problem lies in this function.  You have allocated storage 
  54. on the stack (char str[40]), then return the address of that 
  55. storage.  By the time the function returns, the array on the 
  56. stack no longer exists.  You need persistent storage.
  57.  
  58. One scheme calls for readNextField to allocate memory on the 
  59. heap and return the pointer to that memory.  This introduces an 
  60. ownership problem though, because readNextField allocates but 
  61. doesn't release memory.  readNextField's caller must delete the 
  62. memory it allocated.  This is subject to problems; it is not a 
  63. wise approach.
  64.  
  65. Another scheme calls for allocating storage for readNextField to 
  66. use before calling it, passing a pointer to that storage, and 
  67. letting readNextField populate that storage.  With this 
  68. approach, readNextField must validate the pointer and it must 
  69. assume that there is sufficient space.  This approach is better 
  70. than the first, but it still leaves room for improvement.
  71.  
  72. The other scheme calls for a class to manage the storage, 
  73. returning an object of that class from readNextField.  That is 
  74. the role of a string class.  This approach may be beyond what 
  75. you're ready to do, however.
  76.  
  77. -- 
  78. Robert Stewart        | My opinions are usually my own.
  79. Datalytics, Inc.    | stew@datalytics.com
  80.